19 Kontrola palindromu - #15
Conversation
| Vrátí True, pokud je řetězec palindrom. | ||
| """ | ||
| cislo = len(s) | ||
|
|
There was a problem hiding this comment.
Na tomto řádku máš bílý znak nebo znaky. Pravděpodobně mezery nebo tabulátor. Kvůli tomu ti neprochází kontrola s autopep8
dslavicek
left a comment
dslavicek
left a comment
There was a problem hiding this comment.
V kódu jsem ti zaznačil nevhodně pojmenovanou proměnnou. Tu prosím oprav.
Dále podle zadání má mít tvůj PR jen jeden commit. Po opravě názvu proměnné proto sqashni commity násedujícím způsobem:
Spojení commitu jde udělat přes
git rebase -i <hash commitu předcházejícího všem commitům, které chceš spojovat>(špičatý závorky tam nepatří, těma jen ukazuju kde má být hash). Následně se ti otevře textový editor s commitama a slovama pick u commitů. U commitu který chceš připojit k předchozímu přepíšeš pick na squash, uložíš a zavřeš editor. Potom se ti znova otevře textový editor, tentokrát bude chtít novou commit message. Napíšeš, uložíš, zavřeš a máš sqashnuto. Pushovat je pak potřeba s--force, protože sqashováním měníš historii.Kdyžtak si řekni o pomoc někomu z koučů před lekcí.
| Vrátí True, pokud je řetězec palindrom. | ||
| """ | ||
|
|
||
| cislo = len(s) |
There was a problem hiding this comment.
cislo není vhodný název proměnné. Proměnné prosím pojmenovávej tak, aby šlo poznat k čemu slouží. Tady navhuji například delka_vstupu nebo anglicky input_length.
2 participants
No description provided.